Skip to content

Update to link to ITK Interface Modules#460

Merged
hjmjohnson merged 1 commit into
InsightSoftwareConsortium:mainfrom
blowekamp:main
Jun 13, 2026
Merged

Update to link to ITK Interface Modules#460
hjmjohnson merged 1 commit into
InsightSoftwareConsortium:mainfrom
blowekamp:main

Conversation

@blowekamp

Copy link
Copy Markdown
Member

Removed all occurrences of ITK_USE_FILE and ITK_LIBRARIES.

The new implementation should reduce over-linking and support specific include directories from ITK build trees.

Addresses #459

Removed all occurrences of ITK_USE_FILE and ITK_LIBRARIES.

The new implementation should reduce over-linking and support specific
include directories from ITK build trees.
@blowekamp

Copy link
Copy Markdown
Member Author

There appears to be about a 25% reduction in compilation time on windows.

@dzenanz dzenanz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good on a glance.

@dzenanz

dzenanz commented Jun 12, 2026

Copy link
Copy Markdown
Member

How didn't this make the GitHub any check red?

@hjmjohnson

Copy link
Copy Markdown
Member

This was on my todo list for tomorrow!! Thanks!

@hjmjohnson hjmjohnson merged commit 31de075 into InsightSoftwareConsortium:main Jun 13, 2026
12 checks passed
@blowekamp

Copy link
Copy Markdown
Member Author

How didn't this make the GitHub any check red?

There were some of this type of runtime error which occurred on the first draft. It was addressed by linking to "ITK::ITKFFTImageFilterInit".

This was on my todo list for tomorrow!! Thanks!

I created this as an issue this morning and assigned it to myself to communicate that I was working on it today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants